Skip to content

fix(ci): update Python paths after v1 → archive/v1 reorganization#489

Closed
schwarztim wants to merge 2 commits into
ruvnet:mainfrom
schwarztim:pr/ci-fix-archived-paths
Closed

fix(ci): update Python paths after v1 → archive/v1 reorganization#489
schwarztim wants to merge 2 commits into
ruvnet:mainfrom
schwarztim:pr/ci-fix-archived-paths

Conversation

@schwarztim
Copy link
Copy Markdown
Contributor

Motivation

Commit 81cc241b moved v1/ to archive/v1/. Two CI workflow files still reference the old src/ and tests/ paths, which makes their early steps fail on every push to main and every PR:

  • Code Quality & Security job (ci.yml): Black, flake8, mypy, and bandit all error with Path 'src/' does not exist.
  • Static Application Security Testing (security-scan.yml): bandit and semgrep error similarly.
  • This cascades into Tests (3.10) / (3.11) / (3.12) getting CANCELLED via job dependencies.

Result: every PR shows ~10 failed checks regardless of correctness.

Changes

  • .github/workflows/ci.yml — point Python tooling at archive/v1/src/ + archive/v1/tests/
  • .github/workflows/security-scan.yml — point bandit/semgrep at archive/v1/src/
  • CHANGELOG entry under [Unreleased]

Verification

Workflows are YAML-only changes; structural correctness is the test. After merge:

  • main branch CI should turn green
  • All currently-failing PRs should re-run and turn green (or surface real issues, currently masked)

Note: this PR does not touch any application code.

Copy link
Copy Markdown
Owner

@ruvnet ruvnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Necessary plumbing fix — without this, the Python CI jobs fail looking for /src and /tests at root after the v1 → archive/v1 archive move. Diff is mechanical: each workflow gets the path prefix updated, no behavioural change. Merging.

@ruvnet
Copy link
Copy Markdown
Owner

ruvnet commented May 17, 2026

Closing as superseded — the same archive/v1 path correction was applied to .github/workflows/ci.yml and security-scan.yml on main (with added continue-on-error: true on each step for resilience to optional-dep failures during transition). The diff this PR would land is now empty after rebase. Thanks for catching it.

@ruvnet ruvnet closed this May 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants